Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added IE 187(Maximum Wait Time) and 188(Millisecond Time Stamp) to gtp_v2.py #4376

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mlugo-apx
Copy link

Adds to the GTPv2 protocol capability of Scapy by adding support for the Information Elements 187(Maximum Wait Time) and 188(Millisecond Time Stamp).

Copy link

codecov bot commented May 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.72%. Comparing base (8cea357) to head (766f3f5).
Report is 144 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4376      +/-   ##
==========================================
- Coverage   82.21%   78.72%   -3.50%     
==========================================
  Files         353      334      -19     
  Lines       83586    81044    -2542     
==========================================
- Hits        68723    63802    -4921     
- Misses      14863    17242    +2379     
Files with missing lines Coverage Δ
scapy/contrib/gtp_v2.py 95.48% <100.00%> (+0.06%) ⬆️

... and 291 files with indirect coverage changes

@gpotter2
Copy link
Member

gpotter2 commented May 4, 2024

Thanks for the PR ! Could you add a unit test that builds and/or dissects those new packets?
Thanks

@mlugo-apx
Copy link
Author

Thanks for the PR ! Could you add a unit test that builds and/or dissects those new packets? Thanks

Is there anywhere that shows how to build the test?

@gpotter2
Copy link
Member

gpotter2 commented May 7, 2024

You can read the doc: https://scapy.readthedocs.io/en/latest/development.html#testing-with-utscapy

but the format is trivial to understand.

You can add them to:
https://github.com/secdev/scapy/blob/master/test/contrib/gtp_v2.uts

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants